maybe move the comment saying sp made the block above the function or the block json

maybe move the comment saying sp made the block above the function or the block json

Will that fix the checks? I know this ain't getting merged without all successful checks

maybe move the comment saying sp made the block above the function or the block json

like this?

maybe move the comment saying sp made the block above the function or the block json

like this?

yes but its not going to fix the prettier check, it will be fixed when merged

maybe move the comment saying sp made the block above the function or the block json

like this?

yes but its not going to fix the prettier check, it will be fixed when merged

I don't think it's getting merged unless all checks are successful but ok

Just in case Garbo sees this, I did give permission for him to use my modified code and make the pr.

pleeeease!?

PLEASE?!?

PLEASE?!?

I srsly want this merged.

PLEASE?!?

I srsly want this either merged.

ME TOO

I NEED IT FOR OSU!3D

I NEED IT FOR OSU!3D

ooh yes.

i'm planning on doing a 3d game with VR support eventually too if I get the chance to.

you can of course use it before it gets merged by just copying and pasting the code for the extension and adding it as a custom one

what bothers me about this implementation is that although its "get frame" which would imply no side effects, it does in fact have the side effect of seeking the video around

and there is a code path inside the Promise that never calls resolve or reject (so the block can get stuck forever), and you don't handle when the browser doesn't give you a 2d rendering context (some safari versions are quite picky about this), and using a promise forces a 1 frame delay every time it runs, even if you just want to get the image that the video is currently showing

I NEED IT FOR OSU!3D

ooh yes.

i'm planning on doing a 3d game with VR support eventually too if I get the chance to.

yoooo me too

and there is a code path inside the Promise that never calls resolve or reject (so the block can get stuck forever), and you don't handle when the browser doesn't give you a 2d rendering context (some safari versions are quite picky about this), and using a promise forces a 1 frame delay every time it runs, even if you just want to get the image that the video is currently showing

i don't get it
but i'll tell sharkpool about it anyway

The person who wrote the code should be sending the pull request...

The person who wrote the code should be sending the pull request...

well he told me to do it if i wanted it that badly

Shark pool telling other people to submit code your code for you wastes your time, my time, and the person who submitted its time.

Shark pool telling other people to submit code your code for you wastes your time, my time, and the person who submitted its time.

I told him to do it if he wants. I personally thought it was fine lmao. Next time ill know what to do

and there is a code path inside the Promise that never calls resolve or reject (so the block can get stuck forever), and you don't handle when the browser doesn't give you a 2d rendering context (some safari versions are quite picky about this), and using a promise forces a 1 frame delay every time it runs, even if you just want to get the image that the video is currently showing

WAKE UP I FIXED I- uhhh I mean SHARKPOOL FIXED IT
please merge this 🥺🥺🥺🥺🥺🥺🥺🥺🙏🙏🙏🙏🙏🙏🙏

#1595

#1595

Does it have the fixed video frame thing

Yes

Yes

If GarboMuffin can't merge this one he'd better merge that one then :)

Well he kinda has to, it also fixes a bug with another block

Yes

If GarboMuffin can't merge this one he'd better merge that one then :)

close this one

Yes

If GarboMuffin can't merge this one he'd better merge that one then :)

close this one

Exactly what I was gonna do, thanks for the reminder